Skip to content

docs: add initial style guide docs sections#8161

Draft
joshblack wants to merge 2 commits into
mainfrom
feat/update-style-guide
Draft

docs: add initial style guide docs sections#8161
joshblack wants to merge 2 commits into
mainfrom
feat/update-style-guide

Conversation

@joshblack

@joshblack joshblack commented Jul 14, 2026

Copy link
Copy Markdown
Member

WIP

@changeset-bot

changeset-bot Bot commented Jul 14, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 872aa08

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions Bot added the staff Author is a staff member label Jul 14, 2026
@joshblack joshblack added skip changeset This change does not need a changelog integration-tests: skipped manually Changes in this PR do not require an integration test and removed staff Author is a staff member labels Jul 14, 2026
@github-actions
github-actions Bot temporarily deployed to storybook-preview-8161 July 14, 2026 15:25 Inactive
@joshblack
Enhance storybook documentation for feature stories
872aa08 
Added guidance for writing feature stories to facilitate easy integration into projects.
siddharthkp
siddharthkp reviewed Jul 16, 2026
View reviewed changes
Comment thread .github/skills/style-guide/docs/component-props.md

- `showSelectAll`
- `showSelectedOptionsFirst`
- `hideTitle`

@siddharthkp siddharthkp Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we have an opinion on which one to prefer based on the default value?

hideTitle: true | undefined (false) vs showTitle: undefined (true) | false

siddharthkp
siddharthkp reviewed Jul 16, 2026
View reviewed changes
Comment thread .github/skills/style-guide/docs/component-props.md
- `description`
- `placeholder`
- `caption`
- `text`

@siddharthkp siddharthkp Jul 16, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

text is the only one here that feels generic/non-semantic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@siddharthkp siddharthkp siddharthkp left review comments
Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

integration-tests: skipped manually Changes in this PR do not require an integration test skip changeset This change does not need a changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joshblack @siddharthkp